[FIX]stock_ux: remove mobile field from picking report#908
[FIX]stock_ux: remove mobile field from picking report#908mav-adhoc wants to merge 1 commit intoingadhoc:19.0from
Conversation
|
|
There was a problem hiding this comment.
Pull request overview
Este PR ajusta el reporte de albarán (picking) del módulo stock_ux para dejar de mostrar el teléfono móvil del destinatario y usar únicamente el teléfono fijo, simplificando la información de contacto impresa.
Changes:
- Se elimina la lógica de fallback a
partner_id.mobileen el bloque “Tel:” del destinatario. - Se elimina la lógica de fallback a
partner_id.parent_id.mobileen el mismo bloque.
| <t class="oe_inline" t-out="picking.partner_id.country_id.name"/> | ||
| <div style="font-weight: normal;">Tel: | ||
| <t t-if="picking.partner_id and (picking.partner_id.phone or picking.partner_id.mobile)"> | ||
| <t t-if="picking.partner_id and (picking.partner_id.phone"> |
There was a problem hiding this comment.
La expresión del primer t-if quedó con paréntesis sin cerrar: t-if="picking.partner_id and (picking.partner_id.phone". Así como está, el XML/QWeb no va a parsear/compilar. Ajustá la condición a algo válido (por ejemplo, quitar el paréntesis y chequear solo picking.partner_id and picking.partner_id.phone).
| <t t-if="picking.partner_id and (picking.partner_id.phone"> | |
| <t t-if="picking.partner_id and picking.partner_id.phone"> |
| <t t-if="picking.partner_id and (picking.partner_id.phone"> | ||
| <t t-if="picking.partner_id.phone"> | ||
| <t t-out="picking.partner_id.phone"/> | ||
| </t> | ||
| <t t-else=""> | ||
| <t t-out="picking.partner_id.mobile"/> | ||
| </t> | ||
| </t> | ||
| <t t-elif="picking.partner_id.parent_id and (picking.partner_id.parent_id.phone or picking.partner_id.parent_id.mobile)"> | ||
| <t t-elif="picking.partner_id.parent_id and picking.partner_id.parent_id.phone"> | ||
| <t t-if="picking.partner_id.parent_id.phone"> | ||
| <t t-out="picking.partner_id.parent_id.phone"/> | ||
| </t> |
There was a problem hiding this comment.
Tras eliminar mobile, los t-if internos son redundantes (el t-if/t-elif externo ya garantiza que existe phone). Esto se puede simplificar a una sola condición por bloque y un único t-out, para reducir anidación y facilitar mantenimiento.
mav-adhoc
force-pushed
the
19.0-h-116249-mav
branch
from
a79b293 to
52f69ea
Compare
|
@roboadhoc r+ bump |
No description provided.